Skip to content

UHF-12522: Show links on TPR errand service even if there is no content in the additional information field#1455

Merged
teroelonen merged 1 commit intomainfrom
UHF-12522
Dec 11, 2025
Merged

UHF-12522: Show links on TPR errand service even if there is no content in the additional information field#1455
teroelonen merged 1 commit intomainfrom
UHF-12522

Conversation

@teroelonen
Copy link
Contributor

@teroelonen teroelonen commented Dec 9, 2025

UHF-12522

What was done

  • Show links on TPR errand service even if there is no content in the additional information field.

How to install

  • Make sure your instance is up and running on latest dev-branch on KYMP.
    • git checkout dev && git pull origin dev
    • make fresh
  • Update the HDBT theme
    • composer require drupal/hdbt:dev-UHF-12522
  • Run code updates
    • make drush-locale-update drush-cr

How to test

  •  Go to page and you should now see there "Lisätiedot ja linkit" accordion that has a link to "Vesihuoltolaki". This is not available on production. If the accordion is still not visible go to TPR integration listing and search for 4045 and 4285 IDs and update the errand services. Had this problem myself: I didn't have the links on my local when I started to fix this.
  • Check that the code follows our standards.
  • Check that the differences in visual regression tests are appropriate.

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

✅ Tests passed! You can check the output here: https://city-of-helsinki.github.io/drupal-hdbt-visual-regression/pull/1455/html_report/

Copy link
Contributor

@khalima khalima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So it was a minor bug. ☺️ 🚀

@teroelonen teroelonen merged commit 91d2d26 into main Dec 11, 2025
6 checks passed
@teroelonen teroelonen deleted the UHF-12522 branch December 11, 2025 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants